filechooserentry: Don't use a magic number

... when we have a proper enum value for a column.

entry: Move freeze/thaw into begin/end_change

entry: Don't try to handle Tab in entry completions

Instead, fall through to the default handler after closing the
completion. This has the advantage of letting the file chooser entry
capture the tab key properly, so one can't accidentally move out of the
entry by pressing tab while the completion is popped up.

I also suspect it fixes bugs with weird tab keys and shift/ctrl
oddities. But who knows...

Updated Spanish translation

tests: Relax a refcount comparison check

We now test for real_refcount >= expected_refcount, because various
parts of the code (a11y, selection, cursor, ...) can and do add
references.

a11y: Don't emit children-changed when nothing changed

When we have 0 columns, no children ever get added or removed.

treeview: Run unref helper for all rows

Don't do shortcuts. Because all rows need to be unreffed.
Introduced in 92929b968bd479e237b8f6dc1686ca953f4d8c5d.

Updates

Fix more linking fallout

We are using xinput and gmodule API in a few places in libgtk.
https://bugzilla.gnome.org/show_bug.cgi?id=665326

Add missing symbols to the docs

Doc format fixes

Remove some accidental additions

These functions were never actually implemented

Cosmetic change

Doc typo fix

Add new symbols

Doc typo fix

Cosmetic changes

gtk-demo: Fix warnings in changedisplay

Use the brute force fix. I'm not really interested in fixing this
properly.

a11y: Remove unused gtk_cell_accessible_set_cell_data()

a11y: Make boolean cell render report its states the new way

a11y: Refactor treeview code even more

- Split out set_cell_data()
- Use it
- Get rid of update_cell_value() function, it's now just 2 calls

a11y: Redo function signature

With the recent changes to gtk_cell_accessible_update_cache(), the
update_cell_value() function now needs less code, too.

a11y: Implement GtkContainerCellAccessible.update_cache

a11y: Move update_cache to GtkCellAccesible

This way, we can call it for container renderers, too.

a11y: Redo cell update function

1) always emit signals
   Previously, newly constructed cells would be told to not emit events.
   However, we can ensure that nothing is connected to the signals, so
   they will not actually emit anything.
2) don't return anything
   The return value is unused anyway.

a11y: Remove unused function

a11y: remove now unused property lists

a11y: Don't copy data between renderers

Now that we don't use custom renderers anymore, we don't need to copy
data between them anymore.

a11y: Use cell renderer creation funcs unconditionally

Also delete the previous "new" functions and actually set the real cell
renderer here.

gtk: Add accessible types for cell renderers

a11y: Use the accessible type

cellrenderer: Add API to keep the accessible type

It's private for now, because we require a GType that isn't exported in
the API yet.

API: cellrenderer: Add a private struct

a11y: Add GtkRendererCellAcessible::renderer

a11y: Remove unused variable

treeview: Send changed signal directly to a11y code

No more signal handler is needed, therefore the code can also get rid of
tracking the treemodel. And we use a faster approach for iterating the
changed cellrenderers: We just iterate all columns instead of over all
cell accessibles, as that number is likely quite a bit smaller.

tests: Add test resetting the model

We first set a NULL model and then reset the old model, just to get the
effect of clearing and then resetting.

We reset the cursor and selection afterwards, so the reset doesn't
destroy all the work we did.

treeview: Emit cursor-changed signal when new model is set

a11y: Improve treeview's add/remove_selection()

a11y: Improve get_selected_rows()

a11y: Improve table_is_row_selected()

treeview: Signal selection changes to the accessible

And another signal gone.

a11y: Emit children-changed properly for treeviews

Fixes the patch reverted in b7e74ef95f1d9cd851fb81a124beca0ca11dad00
properly.

https://bugzilla.gnome.org/show_bug.cgi?id=548782

treeview: Remove unnecessary accessible_expanded()

Instead, just call remove_state().

treeview: Add add() function to accessible

... and use that to emit the insertion signals.

treeview: Set expandable flag from treeview

... instead of trying to update it manually.

a11y: Redo cell accessible action handling

Multiple changes:
- actions are now available unconditionally, but only work in the right
  state. This mirrors other actions.
- instead of adding actions manually, they invoke the action on the
  GtkCellAccessibleParent interface.

Tests have been updated to reflect the changes

a11y: Remove "toggle" action from cell renderers

This is identical to "edit" (even though that name is kinda confusing).

treeview: Set EXPANDED state directly on a11y elements

treeview: Deprecate gtk_tree_view_set_destroy_count_func()

The accessibility code doesn't need it anymore.

a11y: Remove destroy_count_func usage

It's not needed anymore.

a11y: Emit row-deleted signal when rows get deleted

Instead of in the old callbacks.

a11y: Revert 22a47c3dbd7875af10df58feedd81d7b661222b6

The fix is wrong on multiple levels. I'll add a proper fix soon.

https://bugzilla.gnome.org/show_bug.cgi?id=548782

treeview: Don't use signals in a11y row expansion

Instead, add a function and call it directly.

a11y: Add _gtk_tree_view_accessible_add_state()

... and _gtk_tree_view_accessible_remove_state(). They should be called
by gtktreeview.c to notify about changes of state for a row.

a11y: Split out function

I want to use that function when sending notifications. See next
commits.

a11y: Readd FOCUSABLE state

The state was lost in commit 08d86fabc4237092bf04a97795498f2c45575085.

Also order the states that are always set alphabetically.

a11y: Remove old notify about showing/visible

As cells are always visible and showing, there is no need to do complex
notification mechanisms anymore.

a11y: Add _gtk_cell_accessible_state_changed()

It's supposed to replace the old state_set calls.

a11y: Split out renderer<->atk state matching

We now keep a state map. This makes adding new states easier, and
doesn't introduce bugs with state change emissions.

a11y: Don't duplicate information anymore

We do provide the expandable and expanded information via
GtkCellRendererState now so there is no need to use extra flags.

a11y: Set new treeview flags

treeview: Set the new flags when rendering the cell

cellrenderer: Add EXPANDED and EXPANDABLE states

These make more sense as flags than they do as properties of the cell
renderer.

tests: Add a test for latest commit

widget: Don't change sensitive state with insensitve parent

This way child.set_sensitive(True) with an insensitive parent doesn't
cause child to become sensitive.

fontchooserwidget: Handle dpi < 0

Just use 96.0

Add new symbols

iconhelper: treat the empty string as a NULL icon name

Treat the empty string as if it was a NULL icon name instead of
rendering GTK_STOCK_MISSING_IMAGE.
This used to be the previous behavior of GtkCellRendererPixbuf and
applications might still rely on it.

cellrendererpix: save the icon size locally instead of in the helper

We need to save the icon size locally for GtkCellRendererPixbuf, since
its properties are set again every time it's drawn (resetting the icon
size on the helper by means of _gtk_icon_helper_clear() each time).

menuitem: cleanup arrow-spacing fetch code

Refactor get_arrow_size() to return the spacing as well, since basically
we always need both.

menuitem: share code between HFW and non-HFW height requests

The code is very similar; having two slightly different code paths is
bad and can lead to bugs. Refactor the code to use the same height
request function.

menuitem: properly respect padding when drawing the separators

Use the padding when calculating the draw coordinates of the separator.

https://bugzilla.gnome.org/show_bug.cgi?id=666242

menuitem: force odd height if we're not using wide separators

So that the 1px line can receive the same amount of space above and
below.

https://bugzilla.gnome.org/show_bug.cgi?id=666242

menuitem: don't request twice the padding for separator menu items

Padding was requested twice for separator menu items.

https://bugzilla.gnome.org/show_bug.cgi?id=666242

demo: Add test for hide-titlebar-when-maximized

Add an additional option to the appwindow demo to test the new
gtk-hide-titlebar-when-maximized setting.

https://bugzilla.gnome.org/show_bug.cgi?id=665616

window: Add hide-titlebar-when-maximized property

For maximized windows, titlebars cannot be used to reposition or
scale the window, so if an application does not use it to convey
useful information (other than the application name), the screen
space occupied by titlebars could be put to better use.
Add a new window property which requests from the window manager
to hide titlebars when windows are maximized to account for this.

https://bugzilla.gnome.org/show_bug.cgi?id=665616

scale: remove unused variable

widget: Synthesize crossing only if sensitivity changed while propagating state

Add has-origin property for GtkScale

If the scale has an origin (it will have one by default), GtkRange will
render the two sides before/after the current value with different style
classes, making it possible for themes to use different colors and
properties for the two areas.
This was possible in GTK 2 with style details, but got lost during the
road to 3.0.

https://bugzilla.gnome.org/show_bug.cgi?id=665140

wayland: Add dummy implementation of the wl_data_device functionality

Setup listener functions for the drag and drop events as well as the selection
events. Then create and save a data structure representing the data offer from
the other client.

wayland: Add an implementation for GdkAtom

This implementation is based on a hash table. The hard coded GtkSelection
atoms are preloaded into the hash table at the correct values. User generated
atoms start after the last preloaded atom.

wayland: Get the data device during input device setup

The data device is a associated with the input device - fetch the data device
from the data manager for the current input device.

wayland: Retrieve the data device manager object

The data device manager is a global object that provides the support
infrastructure around data devices. These data device objects are the basis
for handling drag and drop as well as selections in Wayland.

wayland: Don't use deprecated function: gdk_window_get_pointer

Replace it with gdk_window_get_device_position.

Updated Slovak translation

print dialog: Fix authentication logic

Reset state of CUPS requests correctly during authentization and
check CUPS requests for errors.
Don't initialize variables holding password and username with empty
string (#664640).

Fix nicks for the window class enum

As pointed out in bug 665999, these were just not right.
Before this commit, the nicks were 'output' and 'only'.
After this commit, they are 'input-output' and 'input-only'.

Fix set_focus_child handling during focus changes

In many cases we used to set focus_child to NULL all the way up
to the top and then to the right value, even if there was
a common ancestor, meaning these see a temporary NULL value for
focus_child. Only when the new focus widgets direct parent was
in the previous ancestor list did we stop early.

This fixes that by always stopping propagation when reaching
the common ancestor.

GtkFrame: Fix a casting in the compute_child_allocation() method

The casting used to calculate the child allocation is confusing MAX().
As a result, width and height end up with negative values, which makes no sense.

https://bugzilla.gnome.org/show_bug.cgi?id=666109

themingengine: split a GtkThemingBackground object

Allows easier modification and general sanity of the CSS background
rendering code.

image: take misc padding into account when calculating x/y draw coords

This fixes the misc-alignment.ui reftest.

image: add gtk_image_get_preferred_size()

Add a function to query the preferred size of the GtkImage, and use it
in the get_preferred_width/height implementations.

treeview: Ensure selection's changed signal

There was a corner case where the changed signal was not emitted.
If rows were built like this:
  1    (not selected)
  + 2  (selected)
  + 3  (not selected)
And row 1 was removed, no signal would be emitted.

I like it when writing tests actually finds bugs that have been around
since 2003 - introduced by 4a03ea2334bde7d2e15d0933c5a78f5a2338a854
actually. :)

treeview: Don't try to optimize away selection_changed signal

We get certain cases, in particular with SELECTION_MULTIPLE, where we
cannot figure out in advance of real_set_cursor() if the selection will
actually change.

tests: Add select/unselect calls

treeview: Add a CURSOR_INVALID flag to set_cursor_row()

This is necessary so the code doesn't attempt to do things with the
previous cursor, which currently cause crashes.

treeview: Replace lots of boolean args with flags

I want to add more and 3 boolean args is essentially unreadable code.

tests: Add sanity checks for selection

We want to know that the selection emits "changed" when it changes.

docs: Update code demo for newer glib